Skip to content

Test new config for documentation - #453

Open
GamelinAl wants to merge 1 commit into
mainfrom
447-docs-rendering
Open

GamelinAl wants to merge 1 commit into
mainfrom
447-docs-rendering

Conversation

@GamelinAl

Copy link
Copy Markdown
Member

My test to solve #447

@TeresiaOlsson

Copy link
Copy Markdown
Member

Just so I have understood the solution correctly:

  • Attributes, properties and methods should have their own docstring.

  • Parameters should be in the class docstring and not in the __init__ docstring since the __init__ docstring is ignored in the docs.

  • And then you have customized the conf and css so it takes the output generated by napoleon and puts everything together so we get the format we want for both the readthedocs and help?

@GamelinAl

Copy link
Copy Markdown
Member Author
  • Attributes, properties and methods should have their own docstring.
  • Parameters should be in the class docstring and not in the __init__ docstring since the __init__ docstring is ignored in the docs.
  • And then you have customized the conf and css so it takes the output generated by napoleon and puts everything together so we get the format we want for both the readthedocs and help?

Yes to all 3 but only RTD is changed here. The help just print the class docstring which is already in the good format

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants